Skip to content

Auto-validation for tracing sample.#508

Merged
artursouza merged 3 commits into
dapr:masterfrom
artursouza:tracing_example_validate
Mar 11, 2021
Merged

Auto-validation for tracing sample.#508
artursouza merged 3 commits into
dapr:masterfrom
artursouza:tracing_example_validate

Conversation

@artursouza

Copy link
Copy Markdown
Contributor

Description

Please explain the changes you've made

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #469

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@codecov

codecov Bot commented Mar 11, 2021

Copy link
Copy Markdown

Codecov Report

Merging #508 (5f379f2) into master (1eff655) will decrease coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #508      +/-   ##
============================================
- Coverage     80.43%   80.29%   -0.14%     
+ Complexity      974      973       -1     
============================================
  Files            88       88              
  Lines          3015     3015              
  Branches        330      330              
============================================
- Hits           2425     2421       -4     
- Misses          424      427       +3     
- Partials        166      167       +1     
Impacted Files Coverage Δ Complexity Δ
sdk/src/main/java/io/dapr/utils/Retry.java 50.00% <0.00%> (-25.00%) 1.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1eff655...5f379f2. Read the comment docs.

@wcs1only wcs1only left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@artursouza artursouza merged commit dcaffe8 into dapr:master Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add auto validation for Tracing Example

2 participants